Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix flatten with default #612

Merged
merged 1 commit into from
Nov 17, 2024
Merged

Fix flatten with default #612

merged 1 commit into from
Nov 17, 2024

Conversation

yukinarit
Copy link
Owner

When a field has both flatten and default attributes, pyserde did not render the code correctly. See #610 for more detailed explanation. This commit fixes that as well as the case with alias attribute. See also test_flatten.py.

Closes #610

When a field has both flatten and default attributes, pyserde did
not render the code correctly. See #610 for more detailed
explanation. This commit fixes that as well as the case with alias
attribute. See also test_flatten.py.

Closes #610
@yukinarit yukinarit added the bug Bug report or fix label Nov 16, 2024
@yukinarit yukinarit marked this pull request as ready for review November 17, 2024 11:28
@yukinarit yukinarit merged commit 3a77152 into main Nov 17, 2024
7 checks passed
@yukinarit yukinarit deleted the fix-flatten-default branch November 17, 2024 11:28
github-actions bot pushed a commit that referenced this pull request Nov 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report or fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deserializing Flattened fields with Defaults don't work together as expected
1 participant